TST: split up pandas/tests/indexing/test_multiindex.py #23912
Conversation
|
Hello @simonjayhawkins! Thanks for submitting the PR.
|
Codecov Report
@@ Coverage Diff @@
## master #23912 +/- ##
=======================================
Coverage 92.31% 92.31%
=======================================
Files 161 161
Lines 51513 51513
=======================================
Hits 47554 47554
Misses 3959 3959
Continue to review full report at Codecov.
|
| with catch_warnings(record=True): | ||
| df.ix[name, 'new_col'] = new_vals | ||
|
|
||
| def test_multiindex_label_slicing_with_negative_step(self): |
| columns=['Test'], index=range(30, 71)) | ||
| tm.assert_frame_equal(result, expected) | ||
|
|
||
| def test_multiindex_symmetric_difference(self): |
There was a problem hiding this comment.
can you move to (new file set_ops)
| expected = df.loc[2000, 1, 6][['A', 'B', 'C']] | ||
| tm.assert_series_equal(result, expected) | ||
|
|
||
| def test_get_loc_single_level(self, single_level_multiindex): |
There was a problem hiding this comment.
there are some loc tests in here
| tm.assert_frame_equal(result, expected) | ||
| tm.assert_frame_equal(result, result2) | ||
|
|
||
| def test_getitem_setitem_slice_integers(self): |
There was a problem hiding this comment.
can you move pure getitem tests to a new file, and pure setitem to a newfile as well, if they are both getitem & setitem, put in the setitem one
| frame.loc[('bar', 'three'), 'B'] = 0 | ||
| assert frame.sort_index().loc[('bar', 'three'), 'B'] == 0 | ||
|
|
||
| def test_int_series_slicing( |
| expected = expected.rename(('routine1', 'result1')) | ||
| tm.assert_series_equal(result, expected) | ||
|
|
||
| def test_mixed_depth_insert(self): |
There was a problem hiding this comment.
insert, difference to set_ops
|
@jreback green! |
|
thanks! |
git diff upstream/master -u -- "*.py" | flake8 --diffonly whole tests moved, tests have not been split up. this and the removal of classes is left for a follow-on PR.